-
Notifications
You must be signed in to change notification settings - Fork 27
add more types #473
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add more types #473
Conversation
b23f3bb to
c805e4d
Compare
RagnarGrootKoerkamp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See tiny fixes & suggestion in last commit.
mpsijm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # organize imports in two blocks, non local and local | ||
| section-order = ["standard-library", "first-party"] | ||
| default-section = "standard-library" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No opinion here / I am fine with whatever. This was just the setting closest to how we already formatted the includes manually
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All right! Then I'll change the settings to default when #474 no longer blocks this. The diff should not be too big, and it's automated anyway 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
feel free to just do it, I think I will just write a regex for the rebase anyways
#102